@DaniPopes this should be complete, though no tests yet — unsure if we need those and what'd be the best approach to write them

should we add some helpers to check whether two ASTs are identical?

I was thinking if there's a method on ast nodes to "peek" into the relevant portion of the source code that would be great too

this should be complete, though no tests yet — unsure if we need those and what'd be the best approach to write them

You can add a -Z flag only used in these tests that re-parses pretty printed code; it should also print it to stdout so it gets checked out in a stdout file
I don't think asserting equality is necessary

I was thinking if there's a method on ast nodes to "peek" into the relevant portion of the source code that would be great too

Please check out the SourceMap APIs, I don't know what this has to do with this PR however

Is this PR pending because of lack of tests? Wanted to help get this PR cross the line

I'm currently in the process of rewriting forge fmt using solar, I believe we will be able to upstream that implementation so closing for now

I'm currently in the process of rewriting forge fmt using solar, I believe we will be able to upstream that implementation so closing for now

Hi! Any progress for the new implementation? Or is it open-sourced?